Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Check vfio DMA allocation size before allocating #27

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

bonkf
Copy link
Contributor

@bonkf bonkf commented Apr 27, 2020

The size parameter to memory_allocate_dma() that gets passed to mmap() could be less than MIN_DMA_MEMORY, yet vfio_map_dma() always maps at least MIN_DMA_MEMORY. This is currently not an issue because MIN_DMA_MEMORY is the size of a standard 4KiB page, but it might be larger in future.

@bonkf bonkf changed the title Check vfio DMA allocation size befor allocating Check vfio DMA allocation size before allocating Apr 27, 2020
@emmericp
Copy link
Owner

@ixy-ci ping

src/libixy-vfio.c Outdated Show resolved Hide resolved
@emmericp
Copy link
Owner

emmericp commented May 3, 2020

@ixy-ci ping

@ixy-ci
Copy link

ixy-ci commented May 3, 2020

pong

@emmericp
Copy link
Owner

emmericp commented May 3, 2020

@ixy-ci test

@ixy-ci
Copy link

ixy-ci commented May 3, 2020

Test failed!

Cause: An OpenStack error occurred: Requested operation has failed: Server 948f6c12-c333-4f69-b6be-3fe2c28c993e got into ERROR state

Repository owner deleted a comment from ixy-ci May 3, 2020
@emmericp
Copy link
Owner

emmericp commented May 3, 2020

@ixy-ci test

@ixy-ci
Copy link

ixy-ci commented May 3, 2020

Test failed!

Cause: An OpenStack error occurred: Requested operation has failed: Server 36130943-f194-49b5-a3da-a5c41dc5dceb got into ERROR state

@emmericp
Copy link
Owner

emmericp commented May 3, 2020

looks like the cloud is full :(

"No valid host was found. There are not enough hosts available."

@emmericp
Copy link
Owner

emmericp commented May 7, 2020

@ixy-ci test

@ixy-ci
Copy link

ixy-ci commented May 7, 2020

Test failed!

Cause: An OpenStack error occurred: Access to the resource is denied: Quota exceeded for instances: Requested 1, but already used 4 of 4 instances

@emmericp
Copy link
Owner

emmericp commented May 7, 2020

@ixy-ci test

@ixy-ci
Copy link

ixy-ci commented May 7, 2020

Test failed!

Cause: An OpenStack error occurred: Requested operation has failed: Server b7385af8-f965-4a23-8bca-f382fd76bff1 got into ERROR state

@emmericp
Copy link
Owner

emmericp commented May 7, 2020

cloud is still full, retried because I got this email earlier today:

In the past few weeks, we have been faced multiple times with the situation
that users were no longer able to start new VMs because of the lack of free
CPUs: the cloud was (and is!) full. To relax this situation, we have looked at
the real hardware utilization and came to the conclusion that we will overbook
the CPUs in order to be able to provide more CPUs for virtual machines than
actually exist - the compute cloud will grow virtually. Since the real CPU
load is lower than 20%, we do not expect any noticeable impact on the
performance of the VMs.

the latest test managed to spawn 2 VMs but failed on the third one, awesome

@ackxolotl
Copy link
Contributor

@ixy-ci test

@ixy-ci
Copy link

ixy-ci commented Oct 13, 2020

Test failed!

Cause: An OpenStack error occurred: Received invalid response: openstack cli failed: Output { status: ExitStatus(ExitStatus(256)), stdout: "", stderr: "The request you have made requires authentication. (HTTP 401) (Request-ID: req-9a711922-7ff3-4d84-9ecb-434b2861af56)\n" }

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants